Skip to content

fix calls to llvm prefix mapping functions to use space-separated opt… #81792

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sina-mahdavi
Copy link

@sina-mahdavi sina-mahdavi commented May 28, 2025

With this PR changing some of the prefix mapping API's in llvm, some build errors are triggered in swift. I fixed the API usage to prevent those build errors so that the llvm PR can pass CI tests.

PR for cross-repo testing: swiftlang/llvm-project#10751

@sina-mahdavi
Copy link
Author

Please test with following pull request:
swiftlang/llvm-project#10751

@swift-ci Please test

@sina-mahdavi sina-mahdavi force-pushed the sina-mahdavi/fix-llvm-prefix-mapping-calls branch from ef91ae4 to e8cf513 Compare May 28, 2025 18:19
@sina-mahdavi
Copy link
Author

Please test with following pull request:
swiftlang/llvm-project#10751

@swift-ci Please test

@cachemeifyoucan
Copy link
Contributor

Please test with following pull request:
swiftlang/llvm-project#10751

@swift-ci Please smoke test macOS platform

@benlangmuir
Copy link
Contributor

Please test with following pull request:
swiftlang/llvm-project#10751

@swift-ci Please smoke test Windows

@cachemeifyoucan
Copy link
Contributor

@swift-ci please test macOS platform

@sina-mahdavi
Copy link
Author

Please test with the following pull request:
swiftlang/llvm-project#10751

@swift-ci please test macOS platform

@sina-mahdavi sina-mahdavi force-pushed the sina-mahdavi/fix-llvm-prefix-mapping-calls branch 2 times, most recently from d7bf642 to 4e3053e Compare June 10, 2025 17:49
@sina-mahdavi sina-mahdavi force-pushed the sina-mahdavi/fix-llvm-prefix-mapping-calls branch from 4e3053e to d0c78c0 Compare June 10, 2025 17:51
@sina-mahdavi
Copy link
Author

Please test with the following pull request:
swiftlang/llvm-project#10751

@swift-ci please test

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants